-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TeaVM Impl #262
TeaVM Impl #262
Conversation
…of having to run a server
src/main/java/com/shapesecurity/salvation2/Values/MediaType.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would like to see a test along the lines of, a JS file which imports the resulting JS, runs the exported methods against a sample policy, and asserts on the resulting strings.
(This will require adding the setup-node action, which can hardcode 20 rather than bothering testing against multiple versions.)
@bakkot What if I did this as part of the csp validator website? I think someone mentioned combing the repositories so it might make sense to do that in the JS ecosystem instead of having to implement something here. |
a40f26b
to
a349f2b
Compare
a0c4562
to
728100e
Compare
Support for TeaVM JS Transpiling.
Many of the changes are related to the regexes. TeaVM does not support named groups, so the groups had to be switched to integer based references.